Conversation
if ('operator' in options) | ||
assertionError.operator = options.operator; | ||
assertionError.message = options.message; | ||
assertionError.stack = new Error().stack; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right I had to return assertionError
} | ||
}, | ||
/** | ||
* The equality assertion tests shallow, coercive equality with `=`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I've fixed that!
As I suggested on the IRC I have modified this pull request to point to a different bug. There for scope of is changed and it only aims to get CommonJS test adapter. |
Also I think that code follows all the conventions, if you see where it's not can you please comment inline or post a line number ? |
// See for details: http://wiki.commonjs.org/wiki/Unit_Testing/1.1 | ||
let Assert = suite.Assert || BaseAssert; | ||
|
||
// Going through each item in the test suit and wrapping it into a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: suit -> suite
Not sure why this pull request does not wants to pick up new changes so I made a new one: |
No description provided.